@artemp any feedback or things you can see that could be improved?

@springmeyer - Only calling potentially 'expensive' intersection if interior ring is not entirely within bounding box sounds good to me as I mentioned. I also believe the same logic were used already in mapnik-vector-tile/node-mapnik - I'm sure I saw similar approach already. I'll comment more if I see any possibilities for improvements.

The one possible improvement I can suggest is that Tippecanoe's analogous check has three cases: entirely within the tile (no clipping needed), entirely outside the tile (immediately discarded), and overlapping the tile edge (needs to be clipped). If there are a lot of features that are entirely outside the clip region, it might help to add the quick-discard case here too.

The one possible improvement I can suggest is that Tippecanoe's analogous check has three cases: entirely within the tile (no clipping needed), entirely outside the tile (immediately discarded), and overlapping the tile edge (needs to be clipped).

Thanks @ericfischer I don't think all these cases are tested and so I can't be sure at all whether the current behavior is correct (not buggy) or optimized for perf. So I think we should revert this work and keep in a PR until we can be sure.

Reverted this landing in master prematurely in #91. Recreated #93 to track getting this landed.